Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Created lambda for delete user #1206

Merged
merged 2 commits into from
Jul 26, 2022
Merged

Created lambda for delete user #1206

merged 2 commits into from
Jul 26, 2022

Conversation

itbel
Copy link
Collaborator

@itbel itbel commented Jul 25, 2022 •
edited
Loading

not yet functional

@itbel itbel marked this pull request as ready for review July 25, 2022 23:05
github-advanced-security[bot]
github-advanced-security bot found potential problems Jul 25, 2022
View reviewed changes
amplify/backend/function/deleteJCUser/src/index.js

const user = await deleteCognitoUser(userID)
console.log({ user })
console.log({ env: process.env })

Check failure

Code scanning / CodeQL

Clear-text logging of sensitive information

Sensitive data returned by [process environment](1) is logged here.
@GeorgeBellTMH
Copy link
Contributor

Doesn't this functionality already exist? It should be available under edit user...

@GeorgeBellTMH
Copy link
Contributor

I had talked with Lucas...is there a ticket for this...I have more details...

@GeorgeBellTMH
Copy link
Contributor

#1207

@GeorgeBellTMH
Copy link
Contributor

Merging this because it's needed to do further updates...but we can probably delete this function

@GeorgeBellTMH GeorgeBellTMH merged commit f451d85 into jesus-collective:master Jul 26, 2022
@itbel itbel deleted the ib-delete-user branch November 30, 2022 13:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@itbel @GeorgeBellTMH